Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Issue #76 #77

Merged
merged 1 commit into from
Dec 21, 2015
Merged

Fix Issue #76 #77

merged 1 commit into from
Dec 21, 2015

Conversation

Thubo
Copy link
Contributor

@Thubo Thubo commented Dec 17, 2015

Fixing the issue as suggested by @sharkannon.

Tested on EL7

# cat /etc/redhat-release 
CentOS Linux release 7.2.1511 (Core)
# rpm -qa | grep -E 'puppet|facter|ruby'
rubygem-json-1.7.7-25.el7_1.x86_64
ruby-irb-2.0.0.598-25.el7_1.noarch
rubygem-bigdecimal-1.2.0-25.el7_1.x86_64
rubygem-io-console-0.4.2-25.el7_1.x86_64
ruby-shadow-2.2.0-2.el7.x86_64
ruby-augeas-0.5.0-1.el7.x86_64
libselinux-ruby-2.2.2-6.el7.x86_64
ruby-libs-2.0.0.598-25.el7_1.x86_64
rubygem-rdoc-4.0.0-25.el7_1.noarch
ruby-2.0.0.598-25.el7_1.x86_64
rubygem-psych-2.0.0-25.el7_1.x86_64
rubygems-2.0.14-25.el7_1.noarch
facter-2.4.4-1.el7.x86_64
puppet-3.8.4-1.el7.noarch`

and EL6 clients.

# cat /etc/redhat-release 
CentOS release 6.7 (Final)
[root@el6testclient01 ~]# rpm -qa | grep -E 'puppet|facter|ruby'
ruby-libs-1.8.7.374-4.el6_6.x86_64
ruby-rdoc-1.8.7.374-4.el6_6.x86_64
ruby-shadow-2.2.0-2.el6.x86_64
ruby-1.8.7.374-4.el6_6.x86_64
ruby-irb-1.8.7.374-4.el6_6.x86_64
rubygems-1.3.7-5.el6.noarch
ruby-augeas-0.4.1-3.el6.x86_64
libselinux-ruby-2.0.94-5.8.el6.x86_64
facter-2.4.4-1.el6.x86_64
rubygem-json-1.5.5-3.el6.x86_64
puppet-3.8.4-1.el6.noarch

@edestecd
Copy link
Contributor

This is really annoying, please merge this ;)

jfryman added a commit that referenced this pull request Dec 21, 2015
@jfryman jfryman merged commit 359cda4 into voxpupuli:master Dec 21, 2015
@jfryman
Copy link
Contributor

jfryman commented Dec 21, 2015

@edestecd Happy Holidays to you too! ❤️

@edestecd
Copy link
Contributor

@jfryman Lol, thank you... Have a swell holiday yourself :)

@jehane jehane mentioned this pull request Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants